-
Notifications
You must be signed in to change notification settings - Fork 10
Sample NLP Rule #7
base: master
Are you sure you want to change the base?
Conversation
Thank you for your pull request and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have you on file. In order for us to review and merge your code, please follow the link above and follow the instructions. If you have questions or believe you've received this message in error, don't hesitate to contact Aaron Williamson, our General Counsel and Director of Governance. CLA has not been signed by users @LRParser |
Thanks @LRParser , your PR looks very interesting! I'll give it a try next week; meanwhile @jonfreedman @mistryvinay , feel free to checkout and comment! |
Will work on agreement this week |
@maoo I don't see any link above - |
Thanks, agreement sent to secretary@symphony.foundation |
Thanks, @LRParser! We've received your agreement. You're good to go. (💯 domain, by the way) |
@LRParser , I finally got the time to review and test this PR. The idea is definitely interesting and I think it would be a good contribution to bot-butler, as a demo to build chat bots that rely on an additional server component, like the nlp-server you built. Before merging this PR, I'd propose the following additions:
Thanks! |
Adding a sample NLP rule. Should have a link in documentation to the related NLP server, at: github.com/LRParser/nlp-server
Please let me know what you think!